Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • Trunk
    • None
    • framework
    • None

    Description

      According to Groovy documentation the following construction:

      CompilerConfiguration conf = new CompilerConfiguration();
      conf.setScriptBaseClass(scriptBaseClass);
      groovyClassLoader = new GroovyClassLoader(GroovyUtil.class.getClassLoader(), conf);
      

      is equivalent to running the a script containing the @BaseScript annotation.

      In org.apache.ofbiz.base.util.GroovyUtil instead of choosing one solution, we are doing both.

      Attachments

        1. OFBIZ-10806.patch
          3 kB
          Mathieu Lirzin

        Activity

          I have included OFBIZ-10806.patch which removes the GroovyInit.groovy script and its usage which like described above is redundant.

          mthl Mathieu Lirzin (Inactive) added a comment - I have included OFBIZ-10806.patch which removes the GroovyInit.groovy script and its usage which like described above is redundant.

          Committed in revision 1854652.

          mthl Mathieu Lirzin (Inactive) added a comment - Committed in revision 1854652.

          People

            mthl Mathieu Lirzin (Inactive)
            mthl Mathieu Lirzin (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: